Skip to content

fix: [UIE-9816] - Empty state does not display correctly when category filter is selected first#13360

Merged
pmakode-akamai merged 2 commits intolinode:developfrom
pmakode-akamai:UIE-9816-fix-marketplace-filters-edge-case
Feb 5, 2026
Merged

fix: [UIE-9816] - Empty state does not display correctly when category filter is selected first#13360
pmakode-akamai merged 2 commits intolinode:developfrom
pmakode-akamai:UIE-9816-fix-marketplace-filters-edge-case

Conversation

@pmakode-akamai
Copy link
Contributor

@pmakode-akamai pmakode-akamai commented Feb 4, 2026

Description 📝

Previously, when a category filter was selected first, the UI would show a blank screen instead of the "No results found" empty state.

This edge case happened because filteredProducts still contained products from other categories, even though the selectedCategory itself had zero products. As a result, the empty state logic was never triggered.

This PR fixes the issue by including the selectedCategory in the initial filtering process (when a category is already selected). This ensures filteredProducts accurately reflects the selected category and correctly displays the empty state when no results are available.

NOTE: A changeset may not be necessary for this fix since the main PR has not been released yet.

Changes 🔄

  • Updated the product filtering logic to include selectedCategory during initial filter processing.
  • Ensured the empty state is displayed when the selected category contains no products.
  • Prevented blank screen scenarios when a category filter is applied first.
  • Add more unit tests

Scope 🚢

Upon production release, changes in this PR will be visible to:

  • All customers
  • Some customers (e.g. in Beta or Limited Availability)
  • No customers / Not applicable

Target release date 🗓️

N/A

Preview 📷

Before (on develop branch) After (on this branch)
Screenshot 2026-02-04 at 1 47 40 PM Screenshot 2026-02-04 at 1 47 14 PM

How to test 🧪

Prerequisites

  • Enable Marketplace v2 feature flag

Reproduction steps

  1. Navigate to Partner Referrals landing page
  2. Select "Networking" category
  3. Type "4" into the search field
  4. ⛔️ Observe a blank page
  5. Select either "SaaS & APIs" or "Agentic Systems" type
  6. ℹ️ Observe that the "No results found" message appears
  7. Select "Data Service" type
  8. ⛔️ Observe the "No results found" message disappears and a blank page is shown again

Verification steps

  • Follow the reproduction steps and verify that no blank screen is displayed at any point
    • Verify that when searching for product details that do not belong to the selected category, the "No results found" empty state is displayed instead of a blank screen
  • Ensure all filtering scenarios work as expected
  • Ensure all tests pass
Author Checklists

As an Author, to speed up the review process, I considered 🤔

👀 Doing a self review
❔ Our contribution guidelines
🤏 Splitting feature into small PRs
➕ Adding a changeset
🧪 Providing/improving test coverage
🔐 Removing all sensitive information from the code and PR description
🚩 Using a feature flag to protect the release
👣 Providing comprehensive reproduction steps
📑 Providing or updating our documentation
🕛 Scheduling a pair reviewing session
📱 Providing mobile support
♿ Providing accessibility support

  • I have read and considered all applicable items listed above.

As an Author, before moving this PR from Draft to Open, I confirmed ✅

  • All tests and CI checks are passing
  • TypeScript compilation succeeded without errors
  • Code passes all linting rules

@pmakode-akamai pmakode-akamai marked this pull request as ready for review February 4, 2026 08:36
@pmakode-akamai pmakode-akamai requested a review from a team as a code owner February 4, 2026 08:36
@linode-gh-bot
Copy link
Collaborator

Cloud Manager UI test results

🔺 1 failing test on test run #2 ↗︎

❌ Failing✅ Passing↪️ Skipped🕐 Duration
1 Failing864 Passing11 Skipped40m 43s

Details

Failing Tests
SpecTest
clone-linode.spec.tsCloud Manager Cypress Tests→clone linode » can clone a Linode from Linode details page

Troubleshooting

Use this command to re-run the failing tests:

pnpm cy:run -s "cypress/e2e/core/linodes/clone-linode.spec.ts"

@pmakode-akamai pmakode-akamai added the Add'tl Approval Needed Waiting on another approval! label Feb 4, 2026
Copy link
Contributor

@jdamore-linode jdamore-linode left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks @pmakode-akamai, confirmed this fixes the issue and didn't observe any other issues 👍

@github-project-automation github-project-automation bot moved this from Review to Approved in Cloud Manager Feb 4, 2026
@pmakode-akamai pmakode-akamai added Approved Multiple approvals and ready to merge! and removed Add'tl Approval Needed Waiting on another approval! labels Feb 5, 2026
@pmakode-akamai pmakode-akamai merged commit 02d0ee8 into linode:develop Feb 5, 2026
34 of 35 checks passed
@github-project-automation github-project-automation bot moved this from Approved to Merged in Cloud Manager Feb 5, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Approved Multiple approvals and ready to merge! Marketplace

Projects

Status: Merged

Development

Successfully merging this pull request may close these issues.

4 participants